Skip to content

[helm] Update airflow chart deps to 1.19.0#470

Open
nhuantho wants to merge 2 commits intoopen-metadata:mainfrom
nhuantho:helm-445
Open

[helm] Update airflow chart deps to 1.19.0#470
nhuantho wants to merge 2 commits intoopen-metadata:mainfrom
nhuantho:helm-445

Conversation

@nhuantho
Copy link

What this PR does / why we need it :

Close: #445

Brief change log

  • charts/deps/Chart.yaml: Upgraded Airflow dependency to 1.19.0.
  • Removed webserverSecretKey (previously used for the webserver).
  • Added fernetKey to allow API components to reuse sessions via Fernet encryption.
  • Added apiSecretKey for the API server.
  • Added jwtSecret to support JWT authentication.
  • Removed the webserver configuration, as the new version uses createUserJob for user creation.
  • Added and reorganized component configurations, aligning them with the Airflow values structure to improve usability for end users.

Type of change :

  • Bug fix
  • Improvement
  • New feature
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • Chart Version bumped
  • Variables are documented in the README.md
  • Title of the PR starts with chart name
  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • All new and existing tests passed.

Reviewers

@akash-jain-10
Copy link
Collaborator

Hey @nhuantho . Thanks for the PR. we will review shortly. Meanwhile, could you run the command helm dep up in the directory charts/deps so that the Chart.lock file also gets updated with the above dependency changes!

@nhuantho
Copy link
Author

@akash-jain-10 I ran and updated Chart.lock, can you review for me again?

@akash-jain-10
Copy link
Collaborator

Thanks @nhuantho .

We are reviewing internally to test and validate airflow version 3.1.7. Post that will merge it to be part of next release cycle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update helm chart of airflow to 1.19.0

2 participants